-
-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow import of group and instance issues #3901
base: master
Are you sure you want to change the base?
Conversation
Hey there! Thank you very much for opening this up! Could you check the linting issue? Also: There was something similar implemented in the past, which broke the gitlab integration for a lot of people . Did you test the feature for possible side effects for different configurations? |
Hi, The last commit fixes the linting issue. I tested this with a Installation that was created before the patch and it seemd to be working. Then I changed the new setting and got results from other gitlab projects. I also changed it back to the default and only got the issues from the project. Any hints on what additional checks I could make are greatly welcome. While testing I also found that the link to the issue in the search result was broken and fixed that. I think the last change that also broke it for some people was by me and you are rightfully critical of this change. But because of the new structure of the gitlab implementation this cange is a lot smaller and I think therefore better to review. The only thing I am not sure about are automated tests. I have not yet written tests for angular. |
Sorry for the delay. This slipped under my radar somehow. I just tested it and this breaks using this with gitlab.com (leads to a 500 error). |
Description
This allows searching and importing of gitlab issues on a group and instance level.
Issues Resolved
#2529
Check List